backend/src/test/java/org/sejongisc/backend/betting/service/BettingServiceTransactionalTest.java (1)

60-65: 무료 베팅 테스트 케이스 추가 권장

현재 테스트는 유료 베팅(free=false, stakePoints=100) 시나리오만 다룹니다. 무료 베팅 시나리오에서 포인트 히스토리가 호출되지 않아야 함을 검증하는 테스트 케이스를 추가하는 것이 좋습니다.

backend/src/main/java/org/sejongisc/backend/betting/service/BettingService.java (2)

137-139: 시간 비교 로직 일관성 개선 권장

Line 97에서는 now.isBefore() 및 now.isAfter()를 사용하는 반면, Line 137에서는 now.isAfter()만 사용합니다. 코드 일관성을 위해 동일한 패턴을 사용하는 것이 좋습니다.

-    if (LocalDateTime.now().isAfter(betRound.getLockAt())){
+    LocalDateTime now = LocalDateTime.now();
+    if (now.isAfter(betRound.getLockAt())){
         throw new CustomException(ErrorCode.BET_ROUND_CLOSED);
     }

---

147-147: 포인트 히스토리 originId 일관성 확보
베팅 생성(create) 시 userBetRequest.getRoundId()(라인 107), 취소(cancel) 시 userBet.getUserBetId()(라인 147)를 사용 중입니다. 두 경우 동일한 식별자를 사용하거나 역할을 명확히 문서화하세요.

backend/src/test/java/org/sejongisc/backend/betting/service/BettingServiceTest.java (3)

237-259: 시간 기반 테스트의 안정성 개선을 권장합니다.

LocalDateTime.now()를 사용하면 시간 경계 부근에서 테스트가 불안정해질 수 있습니다.

고정된 시간을 사용하도록 리팩토링하는 것을 권장합니다:

 private BetRound openRoundNow() {
-    LocalDateTime now = LocalDateTime.now();
+    LocalDateTime fixedTime = LocalDateTime.of(2025, 10, 13, 12, 0);
     return BetRound.builder()
             .betRoundID(roundId)
             .scope(Scope.DAILY)
             .status(true)
             .title("OPEN")
-            .openAt(now.minusMinutes(1))
-            .lockAt(now.plusMinutes(10))
+            .openAt(fixedTime.minusMinutes(1))
+            .lockAt(fixedTime.plusMinutes(10))
             .build();
 }

 private BetRound closedRoundNow() {
-    LocalDateTime now = LocalDateTime.now();
+    LocalDateTime fixedTime = LocalDateTime.of(2025, 10, 13, 12, 0);
     return BetRound.builder()
             .betRoundID(roundId)
             .scope(Scope.DAILY)
             .status(true)
             .title("CLOSED")
-            .openAt(now.minusMinutes(10))
-            .lockAt(now.minusMinutes(1))
+            .openAt(fixedTime.minusMinutes(10))
+            .lockAt(fixedTime.minusMinutes(1))
             .build();
 }

---

270-277: 무료 베팅 시 stakePoints 무시 로직 검증을 추가하세요.

Line 275의 주석에서 stakePoints가 무시되어야 한다고 명시하고 있지만, 이를 명시적으로 검증하는 테스트가 없습니다.

postUserBet_free_success 테스트에 다음 검증을 추가하는 것을 고려하세요:

@Test
@DisplayName("postUserBet: 무료 베팅 성공 → 포인트 차감 호출 안함, stake=0")
void postUserBet_free_success() {
    BetRound round = openRoundNow();

    when(betRoundRepository.findById(roundId)).thenReturn(Optional.of(round));
    when(userBetRepository.existsByRoundAndUserId(round, userId)).thenReturn(false);
    when(userBetRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));

    UserBetRequest req = freeReq(); // stakePoints=999 전달

    UserBet result = bettingService.postUserBet(userId, req);

    assertThat(result.isFree()).isTrue();
    assertThat(result.getStakePoints()).isZero(); // ✓ 이미 검증됨
    // 추가: 입력된 999가 무시되고 0으로 설정되었음을 명시적으로 확인
    assertThat(req.getStakePoints()).isEqualTo(999); // 요청 객체는 변경되지 않음
    assertThat(result.getStakePoints()).isNotEqualTo(req.getStakePoints()); // 결과는 다름

    verify(pointHistoryService, never()).createPointHistory(any(), anyInt(), any(), any(), any());
    verify(userBetRepository).save(any(UserBet.class));
}

---

279-364: postUserBet 테스트 커버리지가 우수합니다.

성공 케이스(유료/무료)와 실패 케이스(라운드 없음, 중복, 시간 무효)를 모두 검증하고 있으며, 포인트 서비스 호출 여부를 적절히 확인합니다.

더 엄격한 검증을 위해 저장되는 UserBet 객체의 필드를 명시적으로 확인하는 것을 고려하세요:

@Test
@DisplayName("postUserBet: 유료 베팅 성공 → 포인트 차감 호출 + 저장")
void postUserBet_paid_success() {
    BetRound round = openRoundNow();

    when(betRoundRepository.findById(roundId)).thenReturn(Optional.of(round));
    when(userBetRepository.existsByRoundAndUserId(round, userId)).thenReturn(false);
    when(userBetRepository.save(any())).thenAnswer(inv -> inv.getArgument(0));

    UserBetRequest req = paidReq(100);

    UserBet result = bettingService.postUserBet(userId, req);

    assertThat(result.getStakePoints()).isEqualTo(100);
    assertThat(result.isFree()).isFalse();
    // 추가 검증
    assertThat(result.getUserId()).isEqualTo(userId);
    assertThat(result.getRound()).isEqualTo(round);
    assertThat(result.getOption()).isEqualTo(BetOption.RISE);

    verify(pointHistoryService).createPointHistory(
            eq(userId), eq(-100),
            eq(PointReason.BETTING),
            eq(PointOrigin.BETTING),
            eq(roundId)
    );
    
    // ArgumentCaptor를 사용한 더 엄격한 검증
    ArgumentCaptor<UserBet> captor = ArgumentCaptor.forClass(UserBet.class);
    verify(userBetRepository).save(captor.capture());
    UserBet saved = captor.getValue();
    assertThat(saved.getStakePoints()).isEqualTo(100);
    assertThat(saved.isFree()).isFalse();
}

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro